[NOT FOR IGNITE] [ADLA - ADLS] - Object refactoring#4594
Closed
ro-joowan wants to merge 3 commits intoAzure:previewfrom
Closed
[NOT FOR IGNITE] [ADLA - ADLS] - Object refactoring#4594ro-joowan wants to merge 3 commits intoAzure:previewfrom
ro-joowan wants to merge 3 commits intoAzure:previewfrom
Conversation
* Based on this Swagger PR: Azure/azure-rest-api-specs#1452 * Based on this .NET PR: Azure/azure-sdk-for-net#3512 * Besides code refactoring for type changes, the only real change is updating adapting to how Job submission/build works in src\ResourceManager\DataLakeAnalytics\Commands.DataLakeAnalytics\Commands\SubmitAzureRmDataLakeAnalyticsJob.cs * For the deleted PS<some properties object>.cs files, it is because via Swagger these properties have been flattened. E.g. in src\ResourceManager\DataLakeAnalytics\Commands.DataLakeAnalytics\Models\PSDataLakeStoreAccountInfo.cs: [Obsolete("In a future release this object will have all 'Properties' properties flattened and the 'Properties' property will be removed. Until then, nested properies will be duplicated.")] * Updated the tests ever so slightly to make them more reliable. I also ran them live and updated the SessionRecords folder for both ADLA and ADLS
|
@ro-joowan, |
Member
|
@ro-joowan You need to make these changes in the privarte repository if they are meant for Ignite. There are instructions on moving PRs here: https://github.com/Azure/azure-powershell/wiki/Public-to-Private-Migration . Please let us know if you need help with this. |
* This change is already done in the private repo -- making this change here now so that I can build a local msi succesfully
Member
|
@markcowl this PR isn't meant for the Ignite release. |
|
@ro-joowan, |
Member
|
Please open this PR after Ignite. |
Contributor
Author
|
Sounds good -- I'll track it and ping after Ignite. Thank you |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This checklist is used to make sure that common guidelines for a pull request are followed. You can find a more complete discussion of PowerShell cmdlet best practices here.
General Guidelines
Testing Guidelines
Cmdlet Signature Guidelines
ShouldProcessand haveSupportShouldProcess=truespecified in the cmdlet attribute. You can find more information onShouldProcesshere.OutputTypeattribute if any output is produced - if the cmdlet produces no output, it should implement aPassThruparameter.Cmdlet Parameter Guidelines